Skip to content

Conversation

@Ahmed-Labs
Copy link
Member

@Ahmed-Labs Ahmed-Labs commented Nov 3, 2025

Custom tags were being filtered out from the tag options when adding global filters. This PR handles the case where these custom tags don't have value types.

Fixes DAIN-1057

@Ahmed-Labs Ahmed-Labs changed the title fix(dashboards): Fix add global filter tag filtering condition fix(dashboards): Fix add global filter tag filtering condition for custom tags Nov 3, 2025
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 3, 2025
@Ahmed-Labs Ahmed-Labs changed the title fix(dashboards): Fix add global filter tag filtering condition for custom tags fix(dashboards): Fix global filter tag filtering condition Nov 3, 2025
@Ahmed-Labs Ahmed-Labs marked this pull request as ready for review November 3, 2025 18:09
@Ahmed-Labs Ahmed-Labs requested a review from a team November 3, 2025 18:09
@codecov
Copy link

codecov bot commented Nov 3, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
12315 1 12314 10
View the top 1 failed test(s) by shortest run time
AddFilter does not render unsupported filter keys
Stack Traces | 0.18s run time
Error: expect(element).not.toBeInTheDocument()

expected document not to contain element, found <p class="e1b0h4f90 css-17bnw9v-Label-overflowEllipsis-withChonk e1v132jj2" data-test-id="menu-list-item-label" id="react-aria-_r_5j_">unsupported.measurement</p> instead
    at Object.toBeInTheDocument (.../dashboards/globalFilter/addFilter.spec.tsx:97:11)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@linear
Copy link

linear bot commented Nov 3, 2025

@Ahmed-Labs Ahmed-Labs changed the title fix(dashboards): Fix global filter tag filtering condition fix(dashboards): Ensure custom tag options are displayed when adding global filters Nov 3, 2025
@Ahmed-Labs Ahmed-Labs merged commit 9b0e4b2 into master Nov 3, 2025
49 checks passed
@Ahmed-Labs Ahmed-Labs deleted the ahmedmo/fix/dashboard-global-filter-custom-tags branch November 3, 2025 20:19
shashjar pushed a commit that referenced this pull request Nov 4, 2025
…global filters (#102597)

Custom tags were being filtered out from the tag options when adding
global filters. This PR handles the case where these custom tags don't
have value types.

Fixes
[DAIN-1057](https://linear.app/getsentry/issue/DAIN-1057/fix-bug-where-custom-tags-arent-shown-when-adding-global-filters)
priscilawebdev pushed a commit that referenced this pull request Nov 6, 2025
…global filters (#102597)

Custom tags were being filtered out from the tag options when adding
global filters. This PR handles the case where these custom tags don't
have value types.

Fixes
[DAIN-1057](https://linear.app/getsentry/issue/DAIN-1057/fix-bug-where-custom-tags-arent-shown-when-adding-global-filters)
andrewshie-sentry pushed a commit that referenced this pull request Nov 13, 2025
…global filters (#102597)

Custom tags were being filtered out from the tag options when adding
global filters. This PR handles the case where these custom tags don't
have value types.

Fixes
[DAIN-1057](https://linear.app/getsentry/issue/DAIN-1057/fix-bug-where-custom-tags-arent-shown-when-adding-global-filters)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants